Skip to content

Conversation

@cgwalters
Copy link
Collaborator

These are failing due to infra issues apparently, and we have coverage at release time, we really don't
need to build every single PR by default.

These are failing due to infra issues apparently, and
we have coverage at release time, we really don't
need to build every single PR by default.

Signed-off-by: Colin Walters <[email protected]>
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request removes ppc64le COPR builds from presubmit checks to address CI failures caused by infrastructure problems. The change is straightforward and should improve CI reliability. My review includes a suggestion to comment out the disabled build targets instead of deleting them, to preserve the context for future maintenance. This is a minor improvement for long-term code clarity.

@cgwalters cgwalters enabled auto-merge July 24, 2025 17:59
Copy link
Contributor

@jmarrero jmarrero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@cgwalters cgwalters disabled auto-merge July 24, 2025 18:16
@cgwalters cgwalters enabled auto-merge July 24, 2025 18:16
@cgwalters cgwalters disabled auto-merge July 24, 2025 18:17
This is soon to go EOL and I don't think we need
to keep testing it upstream.

In particular the integration tests are super expensive
and slow and this should cut down our usage of them.
@cgwalters cgwalters changed the title ci: Drop ppc64le COPR builds from presubmits ci: Drop ppc64le && Fedora 41 Jul 24, 2025
@cgwalters cgwalters enabled auto-merge July 24, 2025 18:19
@cgwalters
Copy link
Collaborator Author

Added another commit here to drop Fedora 41

@cgwalters cgwalters disabled auto-merge July 24, 2025 18:35
@cgwalters cgwalters merged commit 1b0ecca into bootc-dev:main Jul 24, 2025
6 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants